Skip to content

Updated contributor branch references - 2.0 no longer maintained #2503

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 3, 2013

Conversation

acoulton
Copy link
Contributor

Q A
Doc fix? yes
New docs? no
Applies to 2.1+
Fixed tickets N/A

The current documentation refers throughout to making bugfix and corrections against the 2.0 branch, however this is no longer maintained. Updated with references to 2.1 to make this more obvious to new contributors.

Obviously this would then need to be repeated each time you drop support for an old version - I'm not sure if you want to go down that route. Alternatively perhaps it would be enough to leave the references in the examples but provide an obvious band on the code and documentation pages clearly showing the oldest maintained version number?

The current documentation refers throughout to making bugfix and
corrections against the 2.0 branch, however this is no longer
maintained. Updated with references to 2.1 to make this more
obvious to new contributors.
weaverryan added a commit that referenced this pull request May 3, 2013
Updated contributor branch references - 2.0 no longer maintained
@weaverryan weaverryan merged commit 425ca69 into symfony:2.1 May 3, 2013
@weaverryan
Copy link
Member

Hi Andrew!

Wow, this is perfect! This has been on my list to do for a few weeks, and you nailed it (even the PR image - extra credit for that!). At least for now, I think we should continue to make these changes going forward for each new, deprecated version. If/where it makes sense, we should remove specific versions (e.g. in one spot you removed "2.0" from a URL and replaced it with "current", which was a great idea). This will hopefully minimize the number of changes we need to make. And like I said, changing the image I think is extra credit - I would be ok if this didn't get updated very often :). I've made a note to use your PR as a reference going forward.

Thanks!

@acoulton
Copy link
Contributor Author

acoulton commented May 3, 2013

Hi @weaverryan thanks - glad you're happy with it :)

@acoulton acoulton deleted the updating-contrib-branch-refs branch May 3, 2013 18:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants